Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

grass.jupyter: move save() to BaseSeriesMap class to reduce redundancy #4378

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

petrasovaa
Copy link
Contributor

@petrasovaa petrasovaa commented Sep 24, 2024

This PR replaces TimeSeriesMap.save() and SeriesMap.save() with BaseSeriesMap.save().

As part of this, it also fixes #4375 by unifying the parameters to save(..., font=None, ...).

@petrasovaa petrasovaa added this to the 8.5.0 milestone Sep 24, 2024
@github-actions github-actions bot added Python Related code is in Python libraries notebook labels Sep 24, 2024
@echoix
Copy link
Member

echoix commented Sep 24, 2024

Do you know if the default font is acceptable on both Linux and windows?

@echoix
Copy link
Member

echoix commented Sep 24, 2024

There's still a problem, as we should've removed the xfail of the test save of time series map test.

Part of the log:

`python/grass/jupyter/tests/timeseriesmap_test.py::test_save `XPASS (D...) [ 34%]

Since we should've failed, that means I've should've used the strict option of pytest.

So todo @petrasovaa : remove the xfail decorator in python/grass/jupyter/tests/timeseriesmap_test.py::test_save

wenzeslaus
wenzeslaus previously approved these changes Sep 25, 2024
@petrasovaa
Copy link
Contributor Author

Do you know if the default font is acceptable on both Linux and windows?

Yes, it comes with the Pillow library

@petrasovaa petrasovaa merged commit 8261363 into OSGeo:main Sep 25, 2024
26 checks passed
@petrasovaa petrasovaa deleted the jupyter-save branch September 25, 2024 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libraries notebook Python Related code is in Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] DejaVuSans.ttf default font for grass.jupyter not found on Windows
3 participants